Skip to content

enable on cennznet extension#25

Open
KarishmaBothara wants to merge 23 commits into
masterfrom
useOnlyCENNZnetExtension
Open

enable on cennznet extension#25
KarishmaBothara wants to merge 23 commits into
masterfrom
useOnlyCENNZnetExtension

Conversation

@KarishmaBothara

Copy link
Copy Markdown

No description provided.

Comment thread packages/extension-dapp/src/index.ts Outdated
@KarishmaBothara KarishmaBothara force-pushed the useOnlyCENNZnetExtension branch from eea4ead to db1cfac Compare October 14, 2021 02:50
Comment thread .eslintrc.js Outdated
'**/coverage/*',
'**/node_modules/*'
'**/node_modules/*',
'**/config.ts'

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just checking which file were you planning to exclude?
packages/extension-chains/src/config.ts still included

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

excluded it... had few listing err in it... mostly related to sort-key in types..
but in case of enum we don't really want to sort...hv added at the top of the file /* eslint sort-keys: 0 */

Comment thread .github/workflows/publish.yml Outdated
run: |
echo "//registry.npmjs.org/:_authToken=${NPM_KEY}" > ~/.npmrc
echo 'Publishing to npmjs.org.'
npx lerna exec -- 'cd build && npm publish --access public'

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can add --dry-run if you want to test the CI before publishing

Suggested change
npx lerna exec -- 'cd build && npm publish --access public'
npx lerna exec -- 'cd build && npm publish --access public'

Comment thread .github/workflows/publish.yml Outdated
Comment thread .github/workflows/publish.yml Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants